BUG: Unable to pass additional arguments to resample().apply() #14635
Conversation
sivieri
commented
Nov 10, 2016
sivieri
commented
- closes Unable to pass additional arguments to resample().apply() #14615
| # otherwise the first element of args is "eaten" by grouper in | ||
| # _groupby_and_aggregate. | ||
| if len(args) == 0: | ||
| return self._groupby_and_aggregate(arg, |
There was a problem hiding this comment.
no you need to figure out the reason and not put in hacks
if u can't then can simply construct the args (you make a tuple) and pass to the function
but would like to see where it's not getting passed thru
There was a problem hiding this comment.
Sorry, I disappeared for a long time!
Yes, I know it was a hack and I agree it's not a real solution... but I don't know the internals of the code, so I'm not sure I can figure this out...
The problem in my opinion is the definition of the function itself: it seems like you cannot have a function with a named argument with a default value, immediately followed by a variadic part, because the interpreter has no idea where to put the first argument (in the named one or as the first parameter of the variadic?).
There was a problem hiding this comment.
I think that *args can be removed from _groupby_and_aggregate. this should solve the issue I think. IOW, you can only pass kwargs except for named args. The grouper arg is really internally anyhow.
jreback
changed the title
|
can you rebase / update |
1 similar comment
|
can you rebase / update |
|
closing as stale. please ping if you'd like to continue. |
